Repository navigation
fix(gateway): verify effective provider transport before receiver operations - #5652
Conversation
@coderabbitai full review |
Action not completed
|
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 42 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. |
cr:no-gate@18a6ff3 reason=service-failure CodeRabbit declined this head's review because its included review quota is exhausted; its current-head response establishes the refusal. The reviewed artifact guard admits continuation and no findings were published. I am advancing to the next lane; no green review is claimed. |
@codex review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. Bravo. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Validation at The real provider observation remains unverified: its credential is available only in the protected deployment. Native HTTP controls establish request and failure boundaries but do not establish current provider state. The draft stays pending the narrow maintainer exception for one protected deployment trial; normal CI, queue and automatic recovery remain required. This change keeps transport disabled and does not complete #5448 or #5412. |
At The maintainer has authorized one protected deployment trial of this head in the interactive session. This is the narrow readiness exception for the real provider observation, which requires the protected deployment credential; that user evaluation is still unverified until the trial runs. Promotion and queue checks, current-head validation and automatic recovery remain enforced. Forwarding stays disabled. Enqueueing will wait for the currently running production recovery to settle. This does not complete #5448 or #5412. |
Merged as The published manifest is Separately, the existing live peer-refusal trial passed at |
Why
The gateway's requested transport setting does not prove what the load balancer has applied. Recovery needs that confirmation before changing gateway protection.
What
Require confirmed provider state before receiver changes and successful public verification. Keep forwarding disabled while preparing its later rollout.
Part of #5448